BUG: GH15426 timezone lost in groupby-agg with cython functions #15433
Conversation
Codecov Report
@@ Coverage Diff @@
## master #15433 +/- ##
==========================================
+ Coverage 90.36% 90.37% +<.01%
==========================================
Files 136 136
Lines 49553 49554 +1
==========================================
+ Hits 44780 44782 +2
+ Misses 4773 4772 -1
Continue to review full report at Codecov.
|
| assert_frame_equal(result, expected) | ||
|
|
||
| def test_agg_time_zone_round_trip(self): | ||
| ts = pd.Timestamp("2016-01-01 12:00:00", tz='US/Pacific') |
There was a problem hiding this comment.
add the issue number as a comment
| df = pd.DataFrame({'a': 1, 'b': [ts + timedelta(minutes=nn) | ||
| for nn in range(10)]}) | ||
|
|
||
| result1 = df.groupby('a')['b'].agg(np.min).iloc[0] |
There was a problem hiding this comment.
compare with .min() as well
| # convert to datetime and change timezone | ||
| from pandas import to_datetime | ||
| result = to_datetime(result).tz_localize(dtype.tz) | ||
| result = to_datetime(result).tz_localize('utc') |
There was a problem hiding this comment.
can add addtl tests in pandas/tests/types/test_cast.py fit them in as appropriate. take out this fix temporarily to validate your test cases, then add back to make sure they work.
|
add examples from #10668 as well. (and you can list list issue in the whatsnew too) |
|
can you update |
stephenrauch
force-pushed
the
tz-lost-in-groupby-agg
branch
from
ae0ea81 to
64a84ca
Compare
|
thanks! |
closes pandas-dev#15426 Author: Stephen Rauch <stephen.rauch+github@gmail.com> Closes pandas-dev#15433 from stephenrauch/tz-lost-in-groupby-agg and squashes the following commits: 64a84ca [Stephen Rauch] BUG: GH15426 timezone lost in groupby-agg with cython functions
3 participants
git diff upstream/master | flake8 --diff